-
Notifications
You must be signed in to change notification settings - Fork 1.2k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Derived Telemetry Prototype #7815
base: master
Are you sure you want to change the base?
Conversation
} | ||
} | ||
// Increment suffix after exhausting the alphabet | ||
suffix = (parseInt(suffix, 10) || 0) + 1; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
why wasn't this covered by tests?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
we'd need 26 different parameters before slapping numbers on them. e.g., a,b,..z,a1,b2,c2,etc.
let valueToUse = rangeItems[0]?.key; | ||
if (!valueToUse) { | ||
// if no numeric data type, just use the first one | ||
valueToUse = metaData.valueMetadatas[0]?.key; |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
same here
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
SWGs have numeric data types, so we never fall through here.
<div :class="['c-comps__refs-controls c-cdef__controls', { disabled: !parameters?.length }]"> | ||
<label v-if="isEditing" class="c-toggle-switch"> | ||
<input type="checkbox" :checked="testDataApplied" @change="toggleTestData" /> | ||
<span class="c-toggle-switch__slider" aria-label="Apply Test Data"></span> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this might not be necessary. You may be able to just use the vanilla label and modify your test to be:
page.getByRole('checkbox', { name : Apply Test Data }
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
tried this, but unfortunately didn't work
Describe your changes:
Closes #7823
Add a new object called "Derived Telemetry" that allows telemetry to be used in simple mathematical expressions.
All Submissions:
Author Checklist
type:
label? Note: this is not necessarily the same as the original issue.Reviewer Checklist